Skip to content

boards: arm: stm32f746 with all mpu regions when testing in userspace #64559

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Nov 6, 2023
Merged

boards: arm: stm32f746 with all mpu regions when testing in userspace #64559

merged 1 commit into from
Nov 6, 2023

Conversation

@FRASTM
Copy link
Contributor

@FRASTM FRASTM commented Oct 30, 2023 •
edited
Loading

Disable the quadspi mpu region of the nucleo_f746zg when testing the samples/userspace/shared_mem
or tests/kernel/mem_protect/userspace.
Impact on stm32 CI

The stm32f7 cortex M7 has 8 MPU regions (fixed by MPU HW @0xe000ed90 ) and the one for quadspi

	quadspi_memory: memory@90000000 {
		compatible = "zephyr,memory-region", "mmio-sram";
		reg = <0x90000000 DT_SIZE_M(256)>;
		zephyr,memory-region = "QSPI";
		zephyr,memory-attr = <( DT_MEM_ARM(ATTR_MPU_EXTMEM) )>;
	};

prevents the testcase to PASS.
As example, running the samples/userspace/shared_mem/sample.kernel.memory_protection.shared_mem on nucleo_f746zg:

ENC Thread Created 0x20010c90
E: num_parts of 4 exceeds maximum allowable partitions (3)
ASSERTION FAIL [ret == 0] @ WEST_TOPDIR/zephyr/samples/userspace/shared_mem/src/main.c:144
k_mem_domain_init() on enc_domain failed -22

Linked to #57467

@FRASTM
boards: arm: stm32f746 with all mpu regions when testing in userspace
3c0d8fb 
Disable the  quadspi mpu region of the nucleo_f746zg
when testing the samples/userspace/shared_mem
or tests/kernel/mem_protect/userspace
The stm32f7 cortex M7 has 8 MPU regions and the one for quadspi prevents
the testcase to PASS.

Signed-off-by: Francois Ramu <[email protected]>
@zephyrbot zephyrbot requested review from ceolin and kartben October 30, 2023 12:15
@FRASTM FRASTM added the platform: STM32 ST Micro STM32 label Oct 30, 2023
@FRASTM FRASTM marked this pull request as draft October 30, 2023 12:21
@FRASTM FRASTM marked this pull request as ready for review October 30, 2023 15:01
@carlescufi carlescufi merged commit 0dcaa4c into zephyrproject-rtos:main Nov 6, 2023
@FRASTM FRASTM mentioned this pull request Apr 26, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@erwango erwango erwango approved these changes

@dcpleung dcpleung dcpleung approved these changes

@andyross andyross Awaiting requested review from andyross

@nashif nashif Awaiting requested review from nashif

@ceolin ceolin Awaiting requested review from ceolin

@kartben kartben Awaiting requested review from kartben

@gautierg-st gautierg-st Awaiting requested review from gautierg-st

@Desvauxm-st Desvauxm-st Awaiting requested review from Desvauxm-st

Assignees

@dcpleung dcpleung

Labels

area: Samples Samples area: Userspace Userspace platform: STM32 ST Micro STM32

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@FRASTM @erwango @dcpleung @carlescufi @zephyrbot